x11: Remove an unnecessary check

We are not creating GDK_NOTHING events anymore. Yay

x11: change event translator interface

Make the event translator return a new event, instead of
filling in a half-constructed one.

Update the two implementation in GdkX11Display and
GdkDeviceManagerXI2.

x11: Change the wm protocols filter api

Instead of passing a half-constructed event and expect
it to be filled in, pass the surface as in argument, and
add an out argument for a newly constructed GdkEvent.

x11: Don't pass a GdkEvent to shape cache filters

The filter functions never look at that event, and
we want to get out of the business of passing half-
constructed events around.

x11: Don't pass a GdkEvent to xsettings filters

The filter functions never look at that event, and
we want to get out of the business of passing half-
constructed events around.

events: Drop GDK_DESTROY

No backend is emitting GDK_DESTROY events anymore, so no
need to carry this around.

gtk: Stop handling GDK_DESTROY differently from GDK_DELETE

We don't have child windows anymore, so there is no difference.

x11: Don't generate GDK_DESTROY events

GTK does not differentiate between GDK_DESTROY and GDK_DELETE
anyway.

x11: use event constructors

This is not quite complete and only handles the
simple cases.

broadway: Use event constructors

Convert all of Broadways event handling to use the new
constructors.

wip: Use event constructors in the testsuite

This doesn't work because the constructors aren't exported,
so remove the gestures text temporarily, until we figure
out how to deal with it.

textview: Stop using gdk_event_copy

main: Use constructors instead of gdk_event_copy + rewriting

imcontextsimple: Use event constructors

widget: Port emulation code to constructors

Use the new event constructors when generating events
for emulating presses.

window: Use event constructors

main: Use event constructors

At the same time, stop setting child_surface in
crossing events. Nothing in GTK looks at it.

gdk: Use constructor for grab broken

gdk: Use event constructors for dnd

events: Drop x_root/y_root

We are not using these fields anymore.

broadway: Stop setting x_root/y_root in events

These fields are not used in GTK anymore, and are
going away soon.

x11: Stop setting x_root/y_root in events

These fields are not used by GTK anymore and
will be going away soon.

x11: Stop relying on root coordinates in events

The X11 dnd code was still using x_root/y_root in
GdkEventMotion in a couple of places. Stop doing so,
these field are going away soon.

Remove root coords from the GdkDrop api

wayland: Clean up coordinate handling

Get rid of the get_coordinates function and just
use the structs directly.

wayland: Use event constructors

Convert all of Waylands event handling to use the new
constructors.

gdk: Add event constructors

Add private API to construct events. This is a step towards
making events readonly, and not objects anymore.

The constructors here are sufficient to convert the Wayland
backend over. More may be added for other backends as needed.

Open issues:
 - history

Make a11y testsuite pass

Merge branch 'master' into 'master'

build: gdk/wayland/cursor depends on wayland-client

See merge request GNOME/gtk!1453

build: gdk/wayland/cursor depends on wayland-client

On distros where wayland-client.h might not be directly in /usr/include
we fail to find the correct headers otherwise.

scale: Remove useless extra local variable

scale: Remove unused member

emojichooser: Add style class to emoji toolbar

Closes #1013

filechooserwidget: Fold function into only caller

filechooserwidget: Use a proper GtkPopoverMenu

Instead of a fake one where we add .menu to a normal popover. This makes
the menu look like other context menus.

filechoosererrorstack: Set a layout manager

Otherwise it doesn't know what to do with its child widget.

filechooserwidget: Allow typing a location in recent mode

This is triggered by typing / or hitting Ctrl+L. Since we don't have a
visual indicator for this mode right now anyway, the reason for not
allowing it in recent mode cited in the comment just above the
early-exit is irrelevant.

Closes #2178

filechooserwidget: Remove unused constants

text: Remove gtk_text_get_im_context()

Unused.

docs: Remove focus_tracking.txt

All the info in there is outdated and not useful.

docs: Remove developers.txt

Everything in there has either been removed in the past or is outdated
information.

docs: Remove dnd_internals.txt

This is either already completely outdated or will be soon.

spinbutton: Remove manual queue_draw() calls

Everything is automatically redrawn when changing the value, etc.

popvermenu: Clarify css node docs

We add the .menu css class to the popover main node, not its contents
node.

aboutdialog: Clarify docs and fix preconditions

modelbutton: Un-select on mouse leave

This brings us closer to the old GtkMenuItem behavior and also makes
more sense.

aboutdialog: Replace visited_links GList with GPtrArray

aboutdialog: Use new GtkStackPage API

stack: Add GtkStackPage:visible accessors

Merge branch 'wip/xdg-popup-async-relayout' into 'master'

Synchronous popup layout

See merge request GNOME/gtk!1241

Merge branch 'ci-flatpak' into 'master'

ci: Update flatpak jobs

See merge request GNOME/gtk!1451

ci: Update flatpak jobs

The CI infrastructure was changed to run with fewer privileges, so we
need to adapt our Flatpak jobs to compensate.

See: https://mail.gnome.org/archives/desktop-devel-list/2020-February/msg00055.html

gdk/surface: Replace move_to_rect() with GdkPopupLayout based API

Replace the gdk_surface_move_to_rect() API with a new GdkSurface
method called gdk_surface_present_popup() taking a new GdkPopupLayout
object describing how they should be laid out on screen.

The layout properties provided are the same as the ones used with
gdk_surface_move_to_rect(), except they are now set up using
GdkPopupLayout.

Calling gdk_surface_present_popup() will either show the popup at the
position described using the popup layout object and a new unconstrained
size, or reposition it accordingly.

In some situations, such as when a popup is set to autohide, presenting
may immediately fail, in case the grab was not granted by the display
server.

After a successful present, the result of the layout can be queried
using the following methods:

 * gdk_surface_get_position() - to get the position relative to its
   parent
 * gdk_surface_get_width() - to get the current width
 * gdk_surface_get_height() - to get the current height
 * gdk_surface_get_rect_anchor() - to get the anchor point on the anchor
   rectangle the popup was effectively positioned against given
   constraints defined by the environment and the layout rules provided
   via GdkPopupLayout.
 * gdk_surface_get_surface_anchor() - the same as the one above but for
   the surface anchor.

A new signal replaces the old "moved-to-rect" one -
"popup-layout-changed". However, it is only intended to be emitted when
the layout changes implicitly by the windowing system, for example if
the monitor resolution changed, or the parent window moved.

gdk/surface: Reset position after hiding

The x and y position of GdkSurface is only valid when visible, so reset
them to their inital state after hiding.

wayland: Simplify logic deciding whether to use xdg_popup

From a handful of hueristics and guess-work, to a single
`surface->surface_type == GDK_SURFACE_POPUP`.

wayland: Fix top-most-popup check

We can map a non-grabbing popup wherever, it's just the grabbing
popup-chain that needs to be ensured not to break any ordering rules.

Fix this by managing two lists; one of open popups, and another for
grabbing ones.

gdk: Move GdkGeometry declaration to gdktypes.h

gdk: Move GdkGravity to gdktypes.h

x11: Make gdk_surface_get_position() return relative position

The returned position should be relative to the parent surface, but
GdkSurface::x,y were only managed properly for O-R windows. This makes
it correct for regular windows too.

wayland: Simplify popup parent discovery

Now popups surfaces are always created with the parent set, so we don't
need to implement vorious guess work to try to find what the parent
might be. Remove that code and just use GdkSurface::parent which is
where the parent set during construction ends up at.

wayland: Remove old "gdk-attached-grab-surface" hackery

The corresponding code in gtk/ is long gone, so it's luckly unused these
days.

wayland/surface: Add per surface configuration event queues

Add event queues specifically for surface configuration events
(xdg_surface.configure, xdg_toplevel.configure, xdg_popup.configure etc)
so that a configuration can be completed without having side effects on
other surfaces. This will be used to synchronously configure specific
GdkSurfaces, as is needed by the Gtk layout mechanisms.

gdk/wayland/surface: Use dedicated bool for frame callback freezing

The freezing is conditioned on various state, so lets make the thawing a
bit more robust. Without this there was a risk that we'd thaw too many
times if there was a frame callback requested while the conditions for
the freezing were not met.

wayland: Remove position method split

Position can only be done via gdk_surface_queue_relayout(); the legacy
way is no longer possible, so remove the legacy positioning code.

gdk/surface: Use enum type for surface type

tests/popover: Add check box for switching autohide prop

popover: Unrealize when autohide prop changes

Changing the autohide property means the popover needs to be remapped.
Remapping may need user interaction, so lets just unrealize the popover
if the property changes, forcing the application to remap it should it
be shown again.

Merge branch 'file-filter-name-property' into 'master'

GtkFileFilter: Add a GObject property for the name

See merge request GNOME/gtk!376

Merge branch 'icontheme-api' into 'master'

Icontheme api

Closes #2410

See merge request GNOME/gtk!1449

icontheme: Update the api

Add properties, and use string arrays instead of lists.

Among other things, this renames gtk_icon_theme_list_icons
to gtk_icon_theme_get_icon_names.

Fixes: https://gitlab.gnome.org/GNOME/gtk/issues/2410

Merge branch 'wip/otte/for-master' into 'master'

debug: When debug-printing, treat NULL as the default display

See merge request GNOME/gtk!1450

debug: When debug-printing, treat NULL as the default display

Otherwise we lose debug prints from all code that allows NULL displays.

Merge branch 'wip/otte/dnd' into 'master'

More DND rework

See merge request GNOME/gtk!1445

Merge branch 'dialog-headerbar-type-annotation' into 'master'

GtkDialog: Add `type Gtk.HeaderBar` annotation to headerbar getter return value

See merge request GNOME/gtk!1447

GtkDialog: Add `type Gtk.HeaderBar` annotation to headerbar getter return value

colorbutton: Fix drag source

By running it in the capture phase, it will not be starved for events by
the button's click source and make drags actually start a color drag
operation.

Remove gdk_content_provider_new_with_formats()

See the removal of gdk_content_provider_new_with_callback() for a
rationale.

tests: Modernize testdnd2

dnd: Port the TreeModel machinery to GValue DND

Merge branch 'text-emoji-history' into 'master'

text: Treat Emoji insertion like clipboard

See merge request GNOME/gtk!1444

placessidebar: Modernize DND

No more shooting pointers through pipes

notebook: Use proper DND

Actually use GValues for the DND operation instead of sending GBytes of
pointer addresses through pipes.

This is a bit complicated because we need to special-case rootwindow
drops, because they're handled on the source side, so we need a custom
ContentProvider.

notebook: Move dnd scrolling via arrows to own drop target

This removes excess code from the tab dnd machinery

dragdest: Handle NULL content formats everywhere

NULL means we don't do content formats checks and accept everything.

We use this for notebook arrows or expanders that react to ongoing
drags.

contentprovider: Add a union content provider

text: Treat Emoji insertion like clipboard

Enter the Emoji inseration in the undo history.
Also, stop stashing away the selection when we
pop up the Emoji chooser, and use the selection
as-is when we insert the Emoji.

placessidebar: Don't claim to support text dnd

Because you don't. And never did.

dragdest: Make gtk_drop_target_new() args be transfer full

Saves tons of code.

colorbutton, colorswatch: Simplify DND

- Use GDK_TYPE_RGBA and let GDK do the conversion from
  application-x-color
- Don't do extra mime type checks when accept() does that for us already

dragdest: Simplify function

Make it obvious that the functions checks one condition and then the
other.

dnd: simplify code

The old code did mimetype checks everywhere when type compatibility has
since been moved to the GtkDropTarget::accept signal.

So the code can now just assume a compatible mime type exists.

dnd: Remove gdk_content_provider_new_with_callback()

Content providers are meant to be immutable, apart from very special
cases, but in those cases they need to emit
gdk_content_provider_content_changed().

Having a constructor that just uses a get_func invites abuse of this
by not making developers aware of those requirments.
In fact, all users in GTK failed to do this.

Instead, code should use the GtkDragSource::prepare signal to create
content providers when needed.

The same problem exists with gdk_content_provider_new_with_formats(),
but this commit doesn't touch that.

testdnd3: Use GValue dnd

contentprovider: Add gdk_content_provider_new_typed()

Gets around the boilerplate required to create and initialize a GValue
by having this function doing it via G_VALUE_COLLECT().

wayland: Destroy the offer on gdk_drop_finish()

Otherwise the compositor won't be aware the dnd failed and will in turn
not notify the drag source.

drop: Add a state machine for tracking events

widget: Fix namespacing problem

In Vala and JS at least, gtk_widget_get_css_name() and
gtk_widget_class_get_css_name() are resolved to
GtkWidget.get_css_name().

To avoid this problem, we rename the class version.

widget: Fix typos for css-clases property

Merge branch 'wip/otte/dnd' into 'master'

some DND fixes

See merge request GNOME/gtk!1440

tests: Do proper local DND

Just transfer GValues.